Fix navbar/filter layout bugs and replace native alerts with Toast in Login - #37
Conversation
…overflow Signed-off-by: Tejashwini <deviteja76@gmail.com>
Signed-off-by: Tejashwini <deviteja76@gmail.com>
LGTM! |
…overflow Signed-off-by: Tejashwini <deviteja76@gmail.com>
Signed-off-by: Tejashwini <deviteja76@gmail.com>
LGTM! |
Description
This PR contains two separate, focused commits:
Commit 1: Fix Settings dropdown z-index overlap and Activity Logs filter panel overflow
Fixes #24
position: relativeandzIndex: 1040to<Navbar>inNavBar.jsxso the Settings dropdown reliably renders above page content instead of behind the admin tab bar..activity-filter-panelgrid inActivityLogs.cssfrom a fixed 7-column layout torepeat(auto-fit, minmax(140px, 1fr))so filter fields wrap onto new rows instead of overflowing the card border.Commit 2: Add reusable Toast component and replace native alerts in Login.jsx
Fixes #36
Login.jsxusedwindow.alert()/alert()for all user feedback (8 call sites: empty field validation, login success/failure, forgot password, reset password). Native alerts block the page and can't be styled.Added a new reusable
Toast.jsxcomponent:success/error/infostyling variantsReplaced all 8
alert()calls inLogin.jsxwithshowToast(message, type), backed by local component state and a<Toast />render at the top of the component.Note: this PR also supersedes the earlier closed PR #30, which contained just the CSS fixes for #24 and those changes are now included here as Commit 1, combined with the more substantial Toast implementation as requested.
Before / After
Settings dropdown:


Filter panel overflow:


Native alert → Toast (Login flow):
learnhub.login.flow.mp4
Steps to Test
Build Verification
Ran
npm run devinfrontend. builds and runs cleanly with no new warnings or errors.Type of Change
Checklist